-
Notifications
You must be signed in to change notification settings - Fork 7
홈 화면 개편 3차 #511
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
홈 화면 개편 3차 #511
Conversation
…상 개선(버튼 눌림 효과 딜레이 최소화)
…xt height 달라지는 현상 해결
…화 로직을 preferenceProvider로 이동
1. 토글 버튼 스케일 축소 2. 홈 화면 기능 버튼이 lock 된 경우(잔액합계,지갑목록) 기존 자물쇠 아이콘에서 체크표시로 변경, pressed animation 제거 3. screen spacing을 제외한 전체 영역에 메뉴 버튼이 배치되도록 크기 조정 #377
- 지갑 순서(walletOrder) - 지갑 즐겨찾기 목록(favoriteWalletIds) - 총 잔액에서 제외(excludedFromTotalBalanceWalletIds) - 홈 화면에 표시되는 기능(homeFeatures)
…글이 먹히지 않는 현상 해결, 가짜잔액입력 필드 애니메이션 먹통 현상 복구
…최근거래, 분석 관련) 로드 코드 일부 복구
|
블록 높이 계산이라서 regtest에서 1일 내 트랜잭션인지 확인할 수 없었던 점, 언어 변경 시 홈 화면 편집하기 바텀시트에서 overflow(최근 거래-영어)랑 클릭 표시 뒤로 글자가 가려지는 부분(지갑 목록, 최근 거래-일본어) 외에 기능은 이상 없었습니다. |
[501] Fiat Price 보기 옵션 추가
- 법정화폐 -> 법정 화폐 - 최근 거래 -> 최근 거래\n(24시간)
…rovider에서 수행하도록 리팩토링, preferenceProvider의 facade 패턴을 이용해 접근 가능하도록 구현
| HomeFeature(homeFeatureTypeString: json['homeFeatureTypeString'], isEnabled: json['isEnabled']); | ||
| } | ||
|
|
||
| // RealmHomeFeature 수정 불필요 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
주석 때문에 여쭤보는데, HomeFeature는 Realm DB를 사용하지 않는 방식으로 변경된거죠?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵!
trim.E60756AF-D296-4D7E-8C25-BD49B424014A.MOV네트워크가 꺼진 상태에서 앱 실행 시 계속 로딩 상태로 보이네요. |
|
지갑 추가 후 사용하던 기존 사용자가 해당 버전으로 업데이트 한 경우 분석 탭에 나오는 거래(보내기/받기/셀프보내기) 트랜잭션 건수가 실제랑 다르게 나오는 경우가 발견됨 |
| } | ||
|
|
||
| // 필요한 경우 호출 (날짜 기반으로 조회) | ||
| void getPendingAndRecentDaysTransactions(int? analysisPeriod, int days, {bool forceRefresh = false}) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
함수는 get으로 시작하는데 반환하는 값은 없어서..
가져온 값이 어떻게 사용되는건지 코드가 잘 안읽히네요.


❗️머지되기 전에 #501 브랜치를 먼저 머지해야함
변경사항
추가
변경
#469